Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: disable facets knowledge panels for crawlers #8772

Merged
merged 2 commits into from
Aug 1, 2023
Merged

Conversation

stephanegigandet
Copy link
Contributor

Related to openfoodfacts/facets-knowledge-panels#123 : we don't want to load facets knowledge panels for crawlers.

In the future, we might want to disable other things for crawlers, this PR makes is_crawl_bot available to the template engine (only for tags right now, it should be added for all templates later)

@stephanegigandet stephanegigandet requested a review from a team as a code owner August 1, 2023 10:21
@github-actions github-actions bot added product history We have kept 10 years of product revisions. This is useful to monitor edits & product improvements 🧪 tests Template::Toolkit The templating toolkit used by product opener. The starting point for HTML/JS/CSS fixes. Display labels Aug 1, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #8772 (1fb4bc5) into main (3e92f7e) will decrease coverage by 17.23%.
Report is 1 commits behind head on main.
The diff coverage is 100.00%.

❗ Current head 1fb4bc5 differs from pull request most recent head 005ade8. Consider uploading reports for the commit 005ade8 to get more accurate results

@@             Coverage Diff             @@
##             main    #8772       +/-   ##
===========================================
- Coverage   48.72%   31.50%   -17.23%     
===========================================
  Files         117       64       -53     
  Lines       21911    17653     -4258     
  Branches     4871     4058      -813     
===========================================
- Hits        10677     5562     -5115     
- Misses       9930    11273     +1343     
+ Partials     1304      818      -486     
Files Changed Coverage Δ
lib/ProductOpener/Display.pm 9.88% <100.00%> (-0.08%) ⬇️

... and 82 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@sonarcloud
Copy link

sonarcloud bot commented Aug 1, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Contributor

@raphael0202 raphael0202 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@stephanegigandet stephanegigandet merged commit 730f3a6 into main Aug 1, 2023
15 checks passed
@stephanegigandet stephanegigandet deleted the fkp-crawler branch August 1, 2023 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Display product history We have kept 10 years of product revisions. This is useful to monitor edits & product improvements Template::Toolkit The templating toolkit used by product opener. The starting point for HTML/JS/CSS fixes. 🧪 tests
Projects
Development

Successfully merging this pull request may close these issues.

3 participants